moved to a webhook#13836
Conversation
sayakpaul
left a comment
sayakpaul
left a comment
There was a problem hiding this comment.
I have configured the secrets.
| SIG="sha256=$(openssl dgst -sha256 -hmac "${WEBHOOK_SECRET}" payload.json | awk '{print $NF}')" | ||
|
|
||
| - name: Strip fork-supplied reviewer/agent config | ||
| # ai-reviewer fetches its config (.ai/review-rules.md, .ai/review-tools.json, |
There was a problem hiding this comment.
How does it get fetched then? It's important that these files always get fetched from the upstream main.
There was a problem hiding this comment.
exactly like the github action : the repo is cloned in the githubapp server and it reads it
There was a problem hiding this comment.
For review rules, we always use upstream main right?
There was a problem hiding this comment.
no we may use the forked / PR one. I can change that
There was a problem hiding this comment.
correction: it is the case, but we do get the context-script from the PR/fork, which is a big issue. Fixing
| llm_api_key: ${{ secrets.ANTHROPIC_API_KEY }} | ||
| llm_api_base: https://api.anthropic.com | ||
| llm_model: claude-opus-4-6 | ||
| llm_stream: 'true' | ||
| mention_trigger: '@askserge' |
There was a problem hiding this comment.
How do we configure these things?
There was a problem hiding this comment.
in the admin UI. the webook then picks the right now depending on what org/repo calls the service
sayakpaul
left a comment
sayakpaul
left a comment
There was a problem hiding this comment.
Good to merge after the conflicts are resolved. Excited to try it out.
2 participants
What does this PR do?
Moves the serge bot into using the github app deployed on our infra